PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

with open(file_path, 'r', encoding='utf-8') as file:
    return file.read()

with open(file_path, 'r', encoding='utf-8') as file:
    return file.read()

PR Code Suggestions ✨

Explore these optional code suggestions:

 try:
     with open(file_path, 'r', encoding='utf-8') as file:
         return file.read()
-except FileNotFoundError:
+except (FileNotFoundError, IOError, OSError) as e:
+    get_logger().debug(f"Could not read {file_path}: {str(e)}")
     continue

-                    return file.read()
+                    content = file.read().strip()
+                    if content:
+                        return content

 with open(file_path, 'r', encoding='utf-8') as file:
-    return file.read()
+    content = file.read().strip()
+    if content:
+        return content

• Author self-review: I have reviewed the PR code suggestions, and addressed the relevant ones.

This PR won't be merged.

It's overfitted to a specific external program, and the mechanism is wrong (loading configurations in the wrong place for a specific tool with hard-coded relative paths).

Use the existing configuration files to give extra instructions
https://qodo-merge-docs.qodo.ai/usage-guide/configuration_options/#local-configuration-file

@mrT23
Thank you for your review.
The motivation for this pull request is to reduce the cost of synchronising multiple files with the same content.
What do you think about treating extra_instructions as a jinja2 template?
If it is a template, we can include other files using the include directive in jinja2 specification.

@dotneet, I appreciate the initiative and the PR, but it's a strong no.

I don't think that 'extra instructions' (IDE extension) to cursor are the same as 'extra instructions' the Qodo Merge (Git reviewer)

The extra instructions in general should be tailored to Qodo Merge behaviour and needs:
https://qodo-merge-docs.qodo.ai/tools/improve/#how-to-combine-extra-instructions-and-best-practices
It doesn't make sense to me to combine them

@mrT23
Thank you. I didn't know about best_practices.md.